Skip to content

[MNT] Remove temporary dummy CI jobs from test.yml - #1726

Merged
PGijsbers merged 5 commits into
openml:mainfrom
KomalDeep355:mnt/remove-dummy-ci-jobs
Oct 13, 2026
Merged

PGijsbers merged 5 commits into
openml:mainfrom
KomalDeep355:mnt/remove-dummy-ci-jobs

Conversation

@KomalDeep355

Copy link
Copy Markdown
Contributor

Removes the 3 temporary dummy CI jobs added in #1572 as a workaround when branch protection rules still required jobs that no longer existed:

  • dummy_windows_py_sk024
  • dummy_windows_py_sk023
  • dummy_docker

The prerequisite — removing these from branch protection settings — was
completed by @PGijsbers on Mar 4, 2026 (see #1688).

This is a clean re-attempt after the previous PR was closed due to
unresolved merge conflicts.

Fixes #1573

@KomalDeep355

Copy link
Copy Markdown
Contributor Author

@geetu040 @fkiraly — re-opening the change from #1688 which was closed
due to unresolved merge conflicts. Branch protection has already been
updated by @PGijsbers, so this should be ready to merge. Please review!

@codecov-commenter

codecov-commenter commented Oct 13, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 53.53%. Comparing base (95de965) to head (71a48bc).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1726   +/-   ##
=======================================
  Coverage   53.53%   53.53%           
=======================================
  Files          63       63           
  Lines        5234     5234           
=======================================
  Hits         2802     2802           
  Misses       2432     2432           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@KomalDeep355

Copy link
Copy Markdown
Contributor Author

Hi @PGijsbers, thank you for syncing the branch with main. I noticed that the CI checks are still running and the PR is awaiting an approving review. Could you please let me know if any further changes are needed on my end, or if I should simply wait for the checks and review to be completed?
Thanks for your help!

@PGijsbers
PGijsbers merged commit 54e94a3 into openml:main Oct 13, 2026
24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[MNT] remove no longer existing jobs from required list in CI

3 participants